Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sap_ha_pacemaker_cluster: Corosync Totem platform dictionary #834

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

marcelmamula
Copy link
Contributor

Description:

  • Enhanced corosync totem variables for each platform based on Cloud providers
  • Switch to dictionary, similar to stonith dictionary
  • Added documentation links

Important:

  • Some dictionaries are simplified because there are no differences either between OS or HANA/NWAS - Similar to stonith dictionaries.

Example for AWS, which includes differences for both OS and host type:

# Default corosync options - Platform specific
# These options combined with __sap_ha_pacemaker_cluster_corosync_totem_default (Same options are overwritten).
# Documentation sources:
# HANA:
#   SLES: https://docs.aws.amazon.com/sap/latest/sap-hana/sap-hana-on-aws-cluster-configuration.html#sap-hana-on-aws-create-the-corosync-configuration-file
#         https://documentation.suse.com/sbp/sap-15/html/SLES4SAP-hana-sr-guide-perfopt-15-aws/index.html#id-example-for-etccorosynccorosync-conf
#   RHEL: https://docs.aws.amazon.com/sap/latest/sap-hana/sap-hana-on-aws-cluster-configuration-1.html#sap-hana-on-aws-corosync-increase
# NWAS:
#   SLES: https://docs.aws.amazon.com/sap/latest/sap-netweaver/sap-netweaver-ha-setup.html#associations
#   RHEL: https://docs.aws.amazon.com/sap/latest/sap-netweaver/rhel-sap-netweaver-ha-setup.html#associations
__sap_ha_pacemaker_cluster_corosync_totem_platform_dict:
  redhat_hana:
    options:
      token: 30000

  redhat_nwas:
    options:
      token: 30000

  suse_hana:
    options:
      token: 30000
      consensus: 36000
      token_retransmits_before_loss_const: 6
      clear_node_high_bit: 'yes'

  suse_nwas:
    options:
      token: 30000
      consensus: 36000
      clear_node_high_bit: 'yes'

__sap_ha_pacemaker_cluster_corosync_totem_platform:
  "{{ __sap_ha_pacemaker_cluster_corosync_totem_platform_dict[ansible_os_family | lower ~ '_hana']
    if sap_ha_pacemaker_cluster_host_type | select('search', 'hana') | length > 0
   else __sap_ha_pacemaker_cluster_corosync_totem_platform_dict[ansible_os_family | lower ~ '_nwas'] }}"

@marcelmamula marcelmamula added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 12, 2024
@marcelmamula marcelmamula self-assigned this Aug 12, 2024
Copy link
Contributor

@ja9fuchs ja9fuchs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@marcelmamula marcelmamula removed the request for review from sean-freeman August 15, 2024 11:23
@marcelmamula marcelmamula merged commit d8424a9 into sap-linuxlab:dev Aug 15, 2024
3 of 4 checks passed
@marcelmamula marcelmamula deleted the totem branch August 15, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants